Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add plugin Tab Filter #1195

Merged
merged 2 commits into from Apr 22, 2013
Merged

Add plugin Tab Filter #1195

merged 2 commits into from Apr 22, 2013

Conversation

robinmalburn
Copy link
Contributor

No description provided.

@sentience
Copy link
Collaborator

@robinmalburn a few issues:

  • Your changes to repositories.json use spaces for indenting rather than tabs. Please update your pull request to conform to the current indenting standard for this project.
  • The Main.sublime-menu file in your project points to your package directory under the name tabfilter, but you have specified a package name of "Tab Filter" for your package, which means your package will be installed in a directory with that name. You should update the file paths in your Main.sublime-menu file to match the name that your package will be installed under.
  • Your README doesn’t make clear what your package does. What do you mean by “filter the open tabs in the current widow”? Do you mean it displays a file selection panel like Goto Anything, but it lists only the files open in tabs in the current window?

@robinmalburn
Copy link
Contributor Author

Thanks for the feedback, @sentience, and apologies for the few hiccups with this.

repositories.json has now been switched to tab indentation, and the path references in Main.sublime-menu have been fixed to use Tab Filter now,

You're spot on with your description of the plugin providing "Goto Anything" like functionality, but restricted to open tabs (wish I'd thought to phrase it that way in the first place). I've updated the README to try and make it's function a bit clearer now, too.

Again, thanks for the feedback, and hope these tweaks resolve the issues.

@sentience
Copy link
Collaborator

Looks good! Merging now.

sentience added a commit that referenced this pull request Apr 22, 2013
Add plugin Tab Filter.

Conflicts:
	repositories.json
@sentience sentience merged commit 3834ddd into wbond:master Apr 22, 2013
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants